Skip to content

Implemented dynamic profile integration and smart QR improvements#243

Open
Kaustav2706 wants to merge 5 commits into
Dev-Card:mainfrom
Kaustav2706:feature/profile-linking
Open

Implemented dynamic profile integration and smart QR improvements#243
Kaustav2706 wants to merge 5 commits into
Dev-Card:mainfrom
Kaustav2706:feature/profile-linking

Conversation

@Kaustav2706
Copy link
Copy Markdown

@Kaustav2706 Kaustav2706 commented May 22, 2026

🚀 Summary

Implemented major improvements for DevCard profile management and smart QR functionality, along with multiple UI/UX and backend fixes.

Closes #190

✅ Type of Change

  • Bug fix
  • New feature
  • UI / Design change
  • Documentation

✅ What Changed

  • Added dynamic profile integration support
  • Improved smart QR workflow and profile handling
  • Fixed Light/Dark mode contrast issues
  • Added glassmorphism UI polish across pages
  • Improved responsive behavior for mobile and desktop
  • Fixed DELETE request 400 Bad Request issue
  • Improved API request header/body handling
  • Added flicker-free theme loading
  • Seeded and verified demo profile support
  • Improved validation and overall UI consistency

✅ How to Test

  1. Run the project locally using pnpm dev
  2. Open the DevCard dashboard and test profile operations
  3. Verify theme switching, responsive layouts, and QR/profile rendering
  4. Test deleting profile links and confirm no API errors occur

✅ Verification

  • Verified responsive UI behavior
  • Verified profile rendering
  • Verified Light/Dark mode support
  • Verified QR/profile routes
  • Confirmed delete operations work correctly

Thank you! 🚀

@Kaustav2706
Copy link
Copy Markdown
Author

Kaustav2706 commented May 22, 2026

PR submitted successfully @Harxhit

Implemented the requested profile integration, QR improvements, theme fixes, API handling improvements, and responsive UI enhancements.

Please review when available. Thank you!

@Kaustav2706 Kaustav2706 mentioned this pull request May 22, 2026
Open
@Harxhit Harxhit added the gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. label May 22, 2026
@ShantKhatri ShantKhatri requested a review from Harxhit May 23, 2026 16:44
@ShantKhatri
Copy link
Copy Markdown
Contributor

ShantKhatri commented May 23, 2026

@Harxhit Seems some unusual changes on the schema or DB side. Can you look into this once?

@Kaustav2706
Copy link
Copy Markdown
Author

Kaustav2706 commented May 23, 2026 •
edited
Loading

Hi @Harxhit @ShantKhatri , Resolved merge conflicts with upstream/main and updated the PR successfully.

The branch is now up to date and ready for review. Thank you!

Also, I noticed I was removed from the assignees list during the conflict resolution process. Kindly reassign me to this issue/PR if possible.

Harxhit
Harxhit reviewed May 23, 2026
View reviewed changes
Comment thread apps/backend/prisma/schema.prisma Outdated
Harxhit
Harxhit reviewed May 23, 2026
View reviewed changes
Comment thread apps/backend/src/routes/auth.ts
Harxhit
Harxhit reviewed May 23, 2026
View reviewed changes
Comment thread apps/backend/src/routes/auth.ts Outdated
Harxhit
Harxhit reviewed May 23, 2026
View reviewed changes
Comment thread apps/backend/src/app.ts Outdated
@Kaustav2706
Copy link
Copy Markdown
Author

Kaustav2706 commented May 23, 2026

Okay @Harxhit , I will work on it tomorrow at update. Thanks for letting me know.

@Kaustav2706
Copy link
Copy Markdown
Author

Kaustav2706 commented May 24, 2026 •
edited
Loading

Hi @Harxhit @ShantKhatri , Implemented all requested review changes and updated the PR successfully.

Summary:

  • Reverted database configuration back to PostgreSQL with environment variable support.
  • Added request validation using Zod for bypass authentication.
  • Removed token from response body since it is already handled securely via HTTP-only cookies.
  • Restricted localhost origins to non-production environments only.
  • Fixed backend Vitest mocks and verified all tests pass successfully.

The branch is now updated and ready for review.

Thank you!

@Harxhit Harxhit removed the gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. label May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Harxhit Harxhit Harxhit left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add Profile Linking and Smart QR Support to DevCard

3 participants

@Kaustav2706 @ShantKhatri @Harxhit